gtkapplication: fix inhibit

The big refactor contained a small mistake that broke inhibiting.  Fix it.

Updated POTFILES.in

Refactor GtkApplication

gtkapplication.c has turned into a bit of an #ifdef mess over time, and
many of the current checks are incorrect.  As an example, if you build
Gtk for wayland, and exclude the X11 backend, much of the functionality
required by wayland (such as exporting menu models) will be disabled.

Solve that by introducing a backend mechanism to GtkApplication (named
GtkApplicationImpl) similar to the one in GApplication.  Add backends
for Wayland, X11 and Quartz, with X11 and Wayland sharing a common
'DBus' superclass.

                             GtkApplicationImpl
                                      |
                       /--------------+-------------------\
                       |                                  |
            GtkApplicationImplDBus              GtkApplicationImplQuartz
                       |
           /-----------+-----------------\
           |                             |
  GtkApplicationImplX11      GtkApplicationImplWayland

GtkApplicationImpl itself is essentially a bunch of vfuncs that serve as
hooks for various things that the platform-specific backends may be
interested in doing (startup, shutdown, managing windows, inhibit, etc.)

With this change, all platform specific code has been removed from
gtkapplication.c and gtkapplicationwindow.c (both of which are now free
of #ifdefs, except for a UNIX-specific use of GDesktopAppInfo in
gtkapplicationwindow.c).

Additionally, because of the movement of the property-setting code out
of GtkApplicationWindow, the _GTK_APPLICATION_ID properties (and
friends) will be set on non-GtkApplicationWindows, such as dialogs.

https://bugzilla.gnome.org/show_bug.cgi?id=720550

bloatpad: test inhibit

Add an inhibit for logout when there is context in the text buffer.

Fix up default csd titlebars

We were forgetting to show close buttons, and the spacing
was wrong, too.

Updated Hebrew translation

Signed-off-by: Yosef Or Boczko <yoseforb@gmail.com>

gdkwindow: Handle references in "update_windows" list correctly

Since update_windows list is a static variable in GdkWindow.c which
contains pointers to windows which needs to be updated, it can happen
that it contains a pointer to a window even after quit from a gtk_main().
If another gtk_main() is called in the same process it tries to process
windows in the list which leads to a crash.
Correct reference count handling of added windows prevents such applications
from crash.

https://bugzilla.gnome.org/show_bug.cgi?id=711552

about dialog: add docs for new GtkLicense values

about dialog: differentiate GPL "or later" versions

https://bugzilla.gnome.org/show_bug.cgi?id=720410

GtkHeaderBar: Center buttons

Header bars can get taller than the buttons require, and we don't
want them to extend all the way. Center them instead.

GtkHeaderBar: fix flipping of window controls

We were not keepign the separators at the inside.

printing: Add missing strings back for translation

There are string in gtkprintunixdialog.ui marked for translation
which are not in gtkprintunixdialog.ui.h.

https://bugzilla.gnome.org/show_bug.cgi?id=708907

printing: Don't double free connection to CUPS

Don't call httpClose() on http connections which are created
outside of GtkCupsRequest in GtkCupsRequest's functions.

https://bugzilla.gnome.org/show_bug.cgi?id=720338

ScaleButton: make the popup transient to the button's window

Fixes the popup positioning on wayland.

https://bugzilla.gnome.org/show_bug.cgi?id=720256

GtkIMContextSimple: Update compose table

This adds Multi_key + f + i, as well as other sequences,
mainly Greek.

https://bugzilla.gnome.org/show_bug.cgi?id=710510

gdkwindow: Use GdkWindowHints instead of guint for flags

https://bugzilla.gnome.org/show_bug.cgi?id=720342

Fix a compiler warning

Don't return without a value from a non-void function.

window: Sanitize size hint computation

We don't want the maximum size to be smaller than the minimum size. Not
just because it's wrong but also because when this happens the rest of
GTK gets mighty confused and infloops resizing to min-size and
max-size in turns causing a flickering window. Well, at least if you
run X without a window manager. Or your window manager hasn't finished
starting up.

Private RHEL bug finding this issue:
https://bugzilla.redhat.com/show_bug.cgi?id=1035409

clipboard: Don't crash when data is requested from NULL window

This could happen if data was requested from a separate screen now that
multi-screen is no longer supported.

Ideally, we'd want to support copying to other screens, but that
requires solving in GDK as that's X-specific so cannot be well
abstracted by GDK (without the reintroduction of multiple screens).

https://bugzilla.gnome.org/show_bug.cgi?id=719314

gtk-demo: Make double-click work again

Updated Spanish translation

Make app menu fallback a bit more reliable

If the theme has no 'menu' control in the window decorations,
show the menubar after all.

headerbar: Don't add hidden widgets

This is a much simpler fix.

Revert "GtkHeaderBar: don't show dangling separators"

This reverts commit a33e5748db44fc9f73e6411082a7a6ff7abaab08.

Avoid double app menu fallback

Both GtkApplicationWindow and GtkHeaderBar listen for changes
of the gtk-shell-shows-app-menu setting, so they need to somehow
coordinate who is going to take action and show a fallback.
We prefer the menu button in the title over the menubar, so
let GtkApplicationWindow opt out if it finds that the header bar
has been configured to show window controls.

Drop a reference to nonexisting API

The GtkApplicationWindow documentation had a reference
to gtk_header_bar_set_show_fallback_app_menu(). That
function no longer exists, so drop it.

GtkHeaderBar: don't show dangling separators

If we don't have a window icon, we hide the titlebar_icon,
we still add it, so we can't simply go by the number of
children when deciding whether to show the separator or
now. Instead, update the separator visibility as we create
the various buttons.

Make window icons work in GtkHeaderBar

testtitlebar: Set a window icon

This reveals that GtkHeaderBar does not pick up window icons
correctly for it menu button, currently.

gdk: add gdk_window_set_shadow_width()

And deprecate the X11-specific version of it.

We call this new API _set_shadow_width() and not _set_frame_extents()
because we already have a gdk_window_get_frame_extents() with a
different meaning and different type of value.

https://bugzilla.gnome.org/show_bug.cgi?id=720374

headerbar: Respect the show-close-button property

window: Update decorations when updating state

This fixes the maximize button not updating...

testtitlebar: Fix initial values

Make sure that the controls show the initial values fo the
headerbar properties, not the other way around.

testtitlebar: more tests

Turn this into a GtkApplication with an app menu.
Allow to override the gtk-shell-shows-app-menu setting
and the decoration-button-layout style property.

headerbar: Support all kinds of CSD decorations

Move the gtkwindow.c CSD code into GtkHeaderBar, and make it triggerable
by the show-close-button property, and remove shows-fallback-app-menu.

https://bugzilla.gnome.org/show_bug.cgi?id=720233

GtkHeaderBar: treat "" as 'no subtitle'

This is one of the few cases where it makes some sense to blur
the line between and empty string and NULL: without this, it is
hard to reset the subtitle e.g. from a builder file. And we
have the has-subtitle property now to enforce subtitle size
allocation independently.

GtkFileChooser: Don't fill location_entry if we just populated the file list

Only fill the location entry with the file name of the tree view's
selected file when the selection was done by the user.

When the file chooser's action is GTK_FILE_CHOOSER_ACTION_OPEN, it
selects the first file in the tree view once loading has finished. For
this case we don't want it to insert the file name in the location
entry, as it hinders efficient navigation using the location entry. To
achieve this, use a priv flag to keep track of whether the
selection-changed signal was caused by the file chooser itself.

https://bugzilla.gnome.org/show_bug.cgi?id=386569

Make testtitlebar more testy

Now that the nice titlebar example is in gtk3-demo, we can
use testtitlebar as an actual testbed for headerbar stuff.

This immediately reveals size allocation issues when titlebar
widgets change size.

gtk-demo: Use title case consistently

Add a header bar example to gtk-demo

This is basically standalone the testtitlebar example, turned
into a demo.

Updated Spanish translation

testtitlebar: Fix -Wunused-variable warning

GtkWidget *title is no longer used.

https://bugzilla.gnome.org/show_bug.cgi?id=720296

GtkFileChooserDefault: Remove unused struct field

The field changing_folder is no longer used anywhere.

https://bugzilla.gnome.org/show_bug.cgi?id=720293

headerbar: Remove unused private API

Don't remove gtkheaderbarprivate.h, even though it's empty,
since we'll add more to it soon.

https://bugzilla.gnome.org/show_bug.cgi?id=720233

wayland: Fix order of xdg-shell requests

Updated Hebrew translation

Updated Galician translations

Gettext comment for "default:LTR" must be before the string

https://bugzilla.gnome.org/show_bug.cgi?id=720096

Simplify a check

Whenever titlebar is set, title_box is pointing to the
same widget, so we can just always use title_box here.
Pointed out by Jasper St. Pierre.

Trivial typo fix

Use ::has-subtitle in testtitlebar

This also gets simplified.

Use ::has-subtitle when constructing CSD titlebars

GtkWindow was constructing a custom title label to
avoid the size reservation for a subtitle. This is
now easier with the ::has-subtitle property.

GtkHeaderBar: Add a ::has-subtitle property

It is a fairly common case to just want a title, and not
reserve extra space for a subtitle. This is much easier
to get right by setting a boolean property than by
constructing a custom title widget.

Fix a segfault in the event axes demo

Seems that pointer_info can be NULL, after all.

Update headerbar title from window title

This removes a possible source of confusion in our API.
The title is now synchronized both ways between the window
and the headerbar.

https://bugzilla.gnome.org/show_bug.cgi?id=720067

Sync the window title initially

When setting a custom titlebar that happens to be a GtkHeaderBar,
we connect to notify::title to pick up title changes on the headerbar,
but we forgot to sync the title initially. Fix that.

Disconnect signal handlers

We connect to notify::title on app-provided titlebars, but
we forgot to disconnect the signal handler when unsetting
the titlebar.

Add a function to determine the text direction

This function can be used in the rare cases where the locale
has to be changed after gtk_init(). Based on a patch
by Jehan <jehan@girinstud.io>.

https://bugzilla.gnome.org/show_bug.cgi?id=720096

demo: only set transient if demo returns a toplevel

demo: put run button on the left

It is a bit annoying to have to go over to the right side to click it.

wayland: Update to latest xdg-shell.xml

wayland: Make sure to call use_unstable_version

button: Allow :hover on insensitive buttons

You can still hover a mouse on insensitive elements; it's up to the
theme to disable that.
This is in line with the HTML/CSS interpretation of :hover.

Insensitive elements still cannot be clicked.

https://bugzilla.gnome.org/show_bug.cgi?id=719486

gdk: Add some return_if_fail() warnings

so we have a better chance of catching the correct cause of bugs like

https://bugzilla.gnome.org/show_bug.cgi?id=719977

filechooser: Don't crash if invalid thumbnails exist

Failing to load a thumbnail returns a NULL pixbuf. Since the hidpi
patches this wasn't checked when creating the surface. Result: assertion
failure.

https://bugzilla.gnome.org/show_bug.cgi?id=719977

gtkradio*: add (allow-none) annotations were appropriate

https://bugzilla.gnome.org/show_bug.cgi?id=679563

demo: Add demo for advanced event information management

This demo condenses the essentials of advanced management of
input events. Depending on the information available in input events,
this demo will try to represent as much information as possible for
those.

https://bugzilla.gnome.org/show_bug.cgi?id=719987

docs: Add a missing Since: tag

Don't distribute generated sources

This was causing problems when building 3.10.6 against an older
wayland.

GtkMenuItem: only set 'separator' class on the separator line

To make the separator line distinguishable from the widget's normal
border from css.

https://bugzilla.gnome.org/show_bug.cgi?id=719861

Fix make dist

x11: Handle XINotifyPassiveGrab/Ungrab in focus events

The focus handling code is shared between core and XI2 implementations,
so just handle the extra XI2 types for passive grabs. Those must be dealt
with in the same way than active grabs. Focus events with this crossing
mode could happen currently through the XIGrabFocusIn passive grab.

https://bugzilla.gnome.org/show_bug.cgi?id=719762

x11: Handle all XI2 crossing mode values in switch

This fixes potential assertions if a GTK+ app gets to receive
a XINotifyPassiveGrab/Ungrab pointer crossing event, currently
triggerable by XIGrabEnter passive grabs.

http://bugzilla.gnome.org/show_bug.cgi?id=719762

recentchooserdefault: Clean up load_id when we have nothing left to do

Returning FALSE from a GSourceFunc will implicitly remove it, so we need
to do this in order to make the new g_source_remove happy.

Updated POTFILES.skip

window: enable client side decorations deactivation with wayland backend

Revert "gdkwindow-x11: Don't set WM_TAKE_FOCUS if accept_focus is FALSE"

This reverts commit 809aab2c8ec88c7b10254a948ba8754a766b1f7c.

Bug 712536 - Themes with gap_(start|end)_file but no actual file crash

Code factorization in commit 34fd123 reintroduced bug fixed in 0d396ab
with non-equivalent factorized tests.

treeview: Don't draw a background for the bin window

The bin window's background would have to be drawn in the bin window's
size and inside the pixel cache draw function to not cause transparency
issues.

But because it's unnecessary as the view window draws the same
background, we just skip it.

https://bugzilla.gnome.org/show_bug.cgi?id=709027

Tajik translation updated

gtk-demo: fix colorchooser example

A nested dialog on top of a modal dialog must itself be modal,
otherwise it doesn't get any input.

GDK-Win32: Ensure sources use UNIX line endings

This makes things more consistent with the other sources and makes patches
easier to apply cleanly.

MSVC Projects: Use UNIX Line endings

...for all files except the README.txt and the .sln files, which have to
have DOS/Windows line endings.  This makes application of patches, when
applicable, easier.

file-chooser: add a sort-directories-first gsetting

Add an option to sort folders before files in order to be
more constistent with the file manager.

demo: Fix the size of the run button

demo: make demo windows transient to the main window

This prevents them getting lost behind.

demo: Add a GtkHeaderBar

demo: change application name of inner demo

demo: expand notebook tabs

demo: Port to GtkApplication

And use GtkBuilder to construct interface.

gtk object tests: run under local environment

Make sure that the tests don't access the host's session bus or
installed gsettings schemas.

Also disable tests for some classes that leak a connection to the
session bus.

https://bugzilla.gnome.org/show_bug.cgi?id=711715

toolbar: draw a full box for separators

This allows themes do whatever they want as separators, with
paddings, borders and backgrounds.

If "wide-separators" property is true, then, instead of just draw
a frame, also render its background, and take into account the
padding property for its limits.

https://bugzilla.gnome.org/show_bug.cgi?id=719713

button: Call the new function

This simplifies a lot of code and doesn't require hacks setting
in_button anymore.

button: Split out function

This is in preparation of the following cleanup commits.

button: Don't emit a signal on broken grabs

We don't emit one on grab notify, so we don't want one for broken grabs
either.

gdkwindow: Remove an unused assignment

The show variable is assigned to in both branches of an if block a
little further down.

Found with scan-build.

https://bugzilla.gnome.org/show_bug.cgi?id=712760

gdkwindow: Remove an unused assignment

scale is only used if HAVE_CAIRO_SURFACE_SET_DEVICE_SCALE is defined.

Found with scan-build.

https://bugzilla.gnome.org/show_bug.cgi?id=712760

gtkprintunixdialog: Remove a redundant (GFile != NULL) check

g_file_new_for_uri() is guaranteed to return a non-NULL value, so this
check was redundant, and was confusing the static analyser into
returning a false positive, where it thought the file could be NULL.

https://bugzilla.gnome.org/show_bug.cgi?id=712760

gtktreeselection: Eliminate a dead assignment

This is technically a dead assignment, but is nice to retain for
clarity. Moving it to the variable definition shuts scan-build up.

https://bugzilla.gnome.org/show_bug.cgi?id=712760

gtkthemingengine: Eliminate a dead assignment

This is technically a dead assignment, but is nice to retain for
clarity. Moving it to the variable definition shuts scan-build up.

https://bugzilla.gnome.org/show_bug.cgi?id=712760